Repository navigation
ci: test matrix +26.10, -26.7 - #704
Open
polyglotAI-bot wants to merge 1 commit into
Open
polyglotAI-bot wants to merge 1 commit into
polyglotAI-bot wants to merge 1 commit into
Conversation
Bring the ClickHouse version coverage of the HTTP and TCP suites onto the support policy (3 latest series + 2 latest LTS series + head): 26.10, 26.9, 26.8, 26.3. head stays covered by nightly.yml, and the floating latest entry is kept. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
polyglotAI-bot
requested review from
alex-clickhouse and
mzitnik
as code owners
October 26, 2026 05:36
Contributor
There was a problem hiding this comment.
🔵 Needs a closer look
Workflow validation remains outside the completed automated review scope, so manual confirmation is needed before approval.
No open issues 🎉
What changed in this PR
Updates ClickHouse server-version coverage for both HTTP and native TCP regression suites to match the stated support policy. This is a CI-only maintenance change with no client behavior changes.
Changes:
- Replaces
26.7with quoted"26.10"in both version matrices. - Retains
latest,26.9,26.8, and the26.3support floor.
ClickHouse C# Client Compliance Checklist
| Check | Status | Notes |
|---|---|---|
| Protocol compatibility preserved? | Yes | No protocol changes. |
| Type system changes tested comprehensively? | N/A | No type changes. |
| Async patterns correct? | N/A | No async changes. |
| Existing tests untouched? | Yes | Test implementations unchanged. |
| Connection string backward compatible? | N/A | No connection-string changes. |
| Error messages clear and actionable? | N/A | No message changes. |
| Thread safety reviewed? | N/A | No concurrency changes. |
| Public docs updated for public-facing changes? | N/A | No public-facing behavior changes. |
| Changelog fragment added? | N/A | CI maintenance only. |
| Duplicated client logic updated consistently? | N/A | No client logic changes. |
| File | Description |
|---|---|
.github/workflows/tests.yml |
Replaces HTTP regression coverage of 26.7 with 26.10. |
.github/workflows/tests-tcp.yml |
Applies the same version replacement to TCP regression coverage. |
Review effort: Balanced
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
CI/maintenance chore: bring the ClickHouse version coverage of both test suites onto the
support policy — the 3 latest series plus the last 2 LTS series, plus
head. No clientbehavior changes, so no changelog fragment and no version bump.
Target set derived from ClickHouse
utils/list-versions/version_date.tsv:Changes
.github/workflows/tests.yml(regress-clickhouse, HTTP driver): +26.10, -26.7..github/workflows/tests-tcp.yml(regress-clickhouse-tcp, TCP driver): +26.10, -26.7.Both matrices are now
[latest, 26.10, 26.9, 26.8, 26.3].headstays covered bynightly.yml(HTTP and TCP). The floatinglatestentry is kept; it satisfies no policy slot.Notes
AGENTS.mdneeds no edit."26.10"is quoted, so YAML keeps it a string (unquoted, it parses as the float26.1).Both fixtures parse
CLICKHOUSE_VERSIONwithSystem.Version, so 26.10 orders after 26.9for
ClickHouseFeatureMapandTcpServerFeatures.clickhouse/clickhouse-server:latestcurrently resolves to the same image digest as26.10.Test
[latest, 26.10, 26.9, 26.8, 26.3].docker manifest inspect clickhouse/clickhouse-server:<tag>succeeds for 26.10, 26.9, 26.8,26.3,
latestandhead..github/.🤖 Generated with Claude Code
Note
Low Risk
Workflow-only version matrix tweak with no runtime or client behavior changes.
Overview
Updates the ClickHouse version matrix for HTTP (
tests.yml) and TCP (tests-tcp.yml) regression jobs so CI tracks current support policy: three latest series plus two LTS, alongside the floatinglatesttag.Both
regress-clickhousematrices change from including 26.7 to 26.10, yielding[latest, 26.10, 26.9, 26.8, 26.3]. No application or driver code changes—only which server image tags run in CI.Reviewed by Cursor Bugbot for commit ccd3f41. Bugbot is set up for automated code reviews on this repo. Configure here.